Skip to content

Conversation

Tom-Willemsen
Copy link
Member

@Tom-Willemsen Tom-Willemsen commented Oct 1, 2025

Description of work

Stream bluesky documents to kafka. This enables remote, out-of-process callbacks to trigger from bluesky scans or data-points.

See https://reduce.isis.cclrc.ac.uk/redpanda-console/topics/NDW2922_bluesky?p=-1&s=50&o=-1#messages for an example of msgpack-encoded documents which have been streamed from my dev machine.

This is using the upstream bluesky-kafka integration. It looks like that implementation is somewhat used by one or two facilities, but much less than core bluesky. If the upstream implementation causes a problem later, it would be pretty trivial to write our own (direct) wrapper that forwards messages to confluent-kafka via msgpack directly.

Ticket

None

Labels

  • 'bluesky-Semver-Minor' - New feature

Acceptance criteria

  • Pull request title is understandable for a user (e.g. scientist) reading the release notes. The PR title should be a short description of the change from a user perspective.
  • Pull request has appropriate labels for automatic release-notes generation
  • Bluesky documents streamed to kafka unconditionally
  • Kafka being unavailable does not cause errors or a scan to fail (messages should simply buffer in the producer). e.g. test this by pulling network cable on your computer mid-scan, scan should continue unaffected.

Documentation

Kafka docs added in 'developer' section.

Not expecting this to be user-facing, so no user-facing docs added.

@Tom-Willemsen Tom-Willemsen added the bluesky-Semver-Minor New functionality / back-compatible changes label Oct 1, 2025
Don't kill a scan if kafka fails...
@Tom-Willemsen Tom-Willemsen requested a review from rerpha October 1, 2025 17:22
Copy link
Contributor

@rerpha rerpha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

worth an ADR? maybe not yet as it's really a "because we can" atm?

@pull-request-size pull-request-size bot added size/L and removed size/M labels Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bluesky-Semver-Minor New functionality / back-compatible changes size/L

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

2 participants